GdkScreenX11: Dispose visuals

If we let them hold onto their X resources until finalize,
it is too late.

GdkVisualX11: Add a dispose implementation

We need to be able to drop X resources before finalization.

GdkDeviceManagerXI2: chain up in constructed

Dispose does chain up, and the parent class' dispose expects
that the object has been fully constructed.

csd: Improve window button handling

Group the buttons on each side in a box, and give them different
style classes, so themes can differentiate (this is partially a
workaround for limitations in our css selectors). The boxes also
let us add a margin below the buttons, without affecting the
allocation of the buttons themselves.

csd: use the correct icon name for close button

The icon is called window-close-symbolic.

csd: Use a dedicated style class for window buttons

This allows themes to differentiate between regular buttons
in a title bar, and window buttons.

csd: Improve the window title

Manually create the label, so that we can set its margins.

textiter: doc precision for forward_search() and backward_search()

https://bugzilla.gnome.org/show_bug.cgi?id=703313

Add a titlebar testcase

This is an experiment towards how we want client-side titlebars
to look.

Add a gtk_window_close function

This is useful when adding close buttons in custom titlebars.

testheaderbar: fix a mmemonic

Trivial rewording

Make the listbox demo follow the naming convention of the
other demos.

Fix the build

We have to uglify the deprecation messages to make gtk-doc
handle them. The get turned into XML attributes, and there
is no automatic quoting to protect quotes in the text.

Deprecate GtkImageMenuItem

Use GtkMenuItem for menus with only text or GMenu for those
with icons for "nouns" and just text for "verbs".

Deprecate the gtk stock functions

Make ignored settings be set to false by default

docs: Don't drop xml/gtkstock.xml

menushell: don't activate a menu entry when opening a big menu

If a menu is opened and it doesn't fit entirely below or above
the menu bar, gtk+ will place it on top. The button release will
then activate the popup item that happens to appear under the
cursor. Avoid this by ignoring release events if they originated
in the parent menu bar and the duration of the press was too short.

https://bugzilla.gnome.org/show_bug.cgi?id=703069

build: Fix make dist

docs: Add GtkListBox functions to their section and fix links

https://bugzilla.gnome.org/show_bug.cgi?id=702981

Fix default value of scrolled window placement-set property

To make the testsuite happy.

Disable primary paste by default

Fixup some of the example widgets used in the docs

Deprecate GtkIconFactory, GtkIconSet, GtkIconSource

We want to use GtkIconTheme instead. It is worth noting that
the parts that remain undeprecated are GtkIconSize and gtk_icon_size_lookup.

Don't use a GtkIconSet to pick the logo image

Just use the first pixbuf in the default list.

Deprecate and ignore gtk-icon-sizes setting

We don't have icons rendered at arbitrary sizes and we have better
APIs that aren't restricted to GtkIconSizes.

Remove the rest of gtk-touchscreen-mode

Deprecate and ignore gtk-keynav-cursor-only and gtk-keynav-wrap-around

Deprecate and ignore gtk-menu-bar-accel setting

Use 'F10' by default.

Deprecate and ignore gtk-menu popup and popdown delay settings

Just pick good defaults. The a11y team confirmed that they don't
rely on them.

Deprecate and ignore gtk-scrolled-window-placement

Just use GTK_CORNER_TOP_LEFT and allow it to be changed with
gtk_scrolled_window_set_placement().

Deprecate and ignore gtk-fallback-icon-theme

The standard icon themes have built in fallbacks.

Deprecate and ignore the cursor blink settings

And hardcode them to reasonable values.

Deprecate and ignore gtk-can-change-accels

It is disabled by default

Mark gtk-color-palette as deprecated

It is only used by the deprecated color selector widget.

Deprecate and ignore gtk-enable-mnemonics and gtk-auto-mnemonics" properties

They are both enabled by default.

Deprecate and ignore gtk-menu-images setting

GtkImageMenuItem images can still be enabled selectively by the app author
using the always-show-image property on the item.

Updated POTFILES.skip

wayland: Add missing symbol export

gdk_wayland_window_set_use_custom_surface was not being exported
correctly.

https://bugzilla.gnome.org/show_bug.cgi?id=695861

tests: Ignore deprecation warnings

testsuite: Ignore deprecation warnings

gtk-demo: Fix compile warnings

gtk-demo: Remove unused variable

settings Deprecate properties

Deprecate the properties that were brought back from the dead in the
last patch.

Revert "Remove the show-unicode-menu and show-input-method-menu settings"

Removing object properties is too much of an API break, even for
properties we don't expect to be used outside of GTK itself.

This reverts commit 8b811b623c387b99d42fb065535d5f6a05d52fa5.

Deprecate and ignore gtk-button-images setting

Button images can still be enabled selectively by the app author
using the always-use-image property on the button.

window: Make process_updates faster

Only look at "impl" windows in gdk_window_process_updates_with_mode()
since these are the only ones we care about.  This avoids a lot of
unnecessary calls to g_list_copy() and g_object_ref().

Fix a couple documentation warnings

Start deprecating some GtkStock API

Make the testsuite happy that get gets the default value

Adjust the a11y testsuite to match expected output

Fix some regressions from stock migration

Document GtkTextSearchFlags

Move the doc about the flags from gtk_text_iter_forward_search() to the
enum.

https://bugzilla.gnome.org/show_bug.cgi?id=390048

gdkwindow: Don't recurse into clipped children when updating

We don't want to recurse into children that are clipped, as that is
wasted work. We handle this by moving the empty check to the top
of the function and only using the clipped region everywhere.

Start migrating internals away from GtkStock

Remove more discontinued ChangeLogs

They are all in git.

Don't use stock APIs in the testsuite

Remove most of the stock API usage from the tests

With the exception of GtkActions and the tests for stock items.

menu model: set separator label conditionally

When creating separators we were binding the "label" property on the
tracker to the "label" property on the GtkSeparatorMenuItem.

This was problematic for two reasons.

First, it was pointless.  The section header label will never change.

Second, it was causing problems: doing the binding caused the value to
be initially synced up, even if it was NULL.  Doing this caused
GtkMenuItem to create a GtkAccelLabel and add it as a child, which
prevented the separator from being shown normally.

Change the code a bit so that we just call gtk_menu_item_set_label()
when creating the item, if we find the label to be non-NULL.

Also, show() the separator item at first.  GtkMenu manages visibility of
separators internally, but it seems "more correct" to show it ourselves
at first.

GtkMenuTracker: fix leak when creating submenus

Remove usage of stock APIs in demo

With the exception of GtkAction.

Don't use stock APIs in the bloatpad example

Don't use stock APIs in the widget factory

treeview: Ensure we first validate the visible area when validating

This invariant stopped being guaranteed when we moved the visible area
validation from a high priority idle to a tick callback.

Fixes redrawing bugs like row expanding sometimes not having any visual
effect.

treeview: Reorder code

This just moves a function and is in preparation for the next commit.

treeview: Actually remove presize handler when we presize

Avoids multiple calls to it.

textiter: bug fix in forward_search() and backward_search()

When the search is case sensitive, g_utf8_normalize() is not called, so
the skip_decomp argument of the function forward_chars_with_skipping()
must be FALSE.

To verify that, when searching "éb", the count parameter of
forward_chars_with_skipping() have a different value:
- case sensitive: count = 2
- case insensitive: count = 3 (g_utf8_normalize() has been called)

The commit adds unit tests that didn't pass before, and that now pass
(without known regression, obviously).

https://bugzilla.gnome.org/show_bug.cgi?id=702977

Add missing allow-none annotations

Remove ancient changelogs

They can be retrieved from git.

Fix gtk3-demo MSVC Project

Build the new listbox.c source file, as that file was missed.

tests: Ignore deprecations

... for where there's no replacement.

Optimize gdk_x11_screen_supports_net_wm_hint()

Move the call to gdk_x11_atom_to_xatom_for_display() outside of the
search loop in gdk_x11_screen_supports_net_wm_hint().  In my test case
(running Audacious for about a minute), this reduced the total number of
hash table lookups performed from 370,000 to 230,000.

https://bugzilla.gnome.org/show_bug.cgi?id=702913

GMenuModel -> GtkMenu binding: show submenu items

In the non-submenu case we bind the 'visibility' attribute to the
tracker, which takes care of showing the item.  In the submenu case, we
don't bind all of the properties, so we miss this one.

Deal with it by just show()ing the submenu item.

https://bugzilla.gnome.org/show_bug.cgi?id=702332

GtkFileChooserButton: Also emit 'file-set' when changing the selection through the combo box

https://bugzilla.gnome.org/show_bug.cgi?id=696498

GtkFileChooserEntry: don't inline-complete if the entry doesn't have the focus

When the file chooser changes directories, it tells the GtkFileChooserEntry to
change its base folder (i.e. the folder from which relative pathnames are resolved).
GtkFileChooserEntry then starts loading a GtkFileSystemModel asynchronously.
In the finished_loading_cb(), however, it would always ask the GtkEntryCompletion
to insert the completion prefix, since that finished_loading_cb() is what is also used
while the user is typing *in the entry*.

But when the entry doesn't have the focus (e.g. the user changed directories by double-clicking
on the file list in the file chooser), there's no reason to insert completions at all.

https://bugzilla.gnome.org/show_bug.cgi?id=672271
Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Bump version

3.9.6

a11y: Include testdata in EXTRA_DIST

Otherwise tests don't work from a tarball release.

filechooser: In testfilechooser, be strict about the --action option

It would default to GTK_FILE_CHOOSER_ACTION_OPEN if the user's specified --action could not be
parsed.  I can never remember what the right options are, so make the program bail out
if the user specifies an unrecognized option.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Remove rest of gtk-show-input-method-menu and gtk-show-unicode-menu

They are no longer used.

GtkEntry: ellipsize placeholder text

The placeholder string is truncated when there's no
room for it in entry's width.

So, by ellipsizing it, the user can notice the text
was truncated so that they can workaround that by,
for example, maximizing the window.

https://bugzilla.gnome.org/show_bug.cgi?id=702763

testsuite: Don't use deprecated functions

filechooser: Actually return a value

treeview: Remove duplicate function

GtkFileChooser: Missing hour on today's files for it&hu locales on Win32

The file chooser does not show the hours for files modified today
when using it & hu locales on Win32.  This is due to the %k format conversion
for strftime() that is in Olson's Timezone Package, which msvcrt.dll doesn't have.
Cygwin has it, though.

So for now, we just use a %H format.  While %k would render
six-in-the-morning as " 6" (6 preceded by a space), %H will render it
as "06".

https://bugzilla.gnome.org/show_bug.cgi?id=560337

Fix gtk_text_iter_forward_to_tag_toggle() for end iter

The function must return TRUE only if there is a tag toggle _after_ the
iter, not _at_ the iter. So for the end iter, the function must always
return FALSE.

Add also unit tests for gtk_text_iter_forward_to_tag_toggle().

https://bugzilla.gnome.org/show_bug.cgi?id=691266

filechooser: Fix misleading documentation for gtk_file_chooser_set_preview_widget()

https://bugzilla.gnome.org/show_bug.cgi?id=701174

quartz: remove check for keymap changes from update_keymap()

The function is now only called when the keymap has actually
changed. bug #698183.
(cherry picked from commit e62709da2ee6619f26c8446b545418e5f1b9e151)

quartz: update the keymap only if the input method changed

and not on each keystroke, which for some IMs apparently caused a full
update on each keystroke, not just a check for changes. Patch from
Takuro Ashie, bug #698183.

Improve doc of gtk_text_iter_forward_search()

It was not possible to know if the @limit was for @match_start or
@match_end. It was documented for backward_search(), but not for
forward_search().

https://bugzilla.gnome.org/show_bug.cgi?id=390048

gtkfilechooserbutton: fix documentation typo

https://bugzilla.gnome.org/show_bug.cgi?id=702695

filechooser: In testfilechooser, add buttons to get the selection and the current name

filechooser: Add gtk_file_chooser_get_current_name()

Currently you can only set the current filename, but not
get it. It's useful to be able to get it in save
dialogs, where the user has typed the desired filename
and you are not in a real directory (recent used, for
example).

https://bugzilla.gnome.org/show_bug.cgi?id=702497

IconTheme: Move changed emission to an idle

By delaying the emission to an idle we avoid a lot of tricky
reentrancy issues. For instance, a normal gtk_icon_theme_choose_icon()
call could in very rare cases (when a user updated an icon theme) emit
a signal which could affect the icon currently being looked up.  This
kind of reentrancy is very hard to test against, especially when it is
so rare, so we're better of avoiding it.

There is no real value to get the change signal directly anyway. All
it can do is affect which icon is rendered the next frame, and we will
handle the queued emission before rendering. Not to mention that icon
theme change detection is polled anyway, so it is already delayed.

https://bugzilla.gnome.org/show_bug.cgi?id=694755

[l10n] Updated Italian translation.

IconTheme: Clear caches when reloading theme

When we're reloading the theme in ensure_valid_themes (due to noticing
that a theme dir has changed) we need to also clear the icon cache
as it will not be valid for the new theme.

We already do this in do_theme_change(), but ensure_valid_themes()
was missing this.

https://bugzilla.gnome.org/show_bug.cgi?id=702598

IconCache: Keep a ref on the GtkIconData

The icon data in GttkIconInfo->data is currently owned by the
IconThemeDir->icon_data hashtable. However, on e.g. a theme change
blow_themes() destroys the dirs and thus the data, meaning any
outstanding GtkIconInfo points to stale data.

We solve this by adding a refcount to GtkIconData and reffing it
from GtkIconInfo.

https://bugzilla.gnome.org/show_bug.cgi?id=702598

Updated POTFILES.skip

GtkBuilder: add new constructor APIs

Add new APIs gtk_builder_new_from_{file,resource,string}() and encourage
their use over the older _add_from_...() APIs.

https://bugzilla.gnome.org/show_bug.cgi?id=679930

docs: GtkBuilder: warn about unusual GError use

GtkBuilder returns GError for _add_from_{file,resource,string}(),
implying that the user should be able to recover from these errors.
Mention in the docs that it's unreasonable to try to do this.

https://bugzilla.gnome.org/show_bug.cgi?id=679930